Skip to content

Unittest and socket collection#429

Merged
thiagoftsm merged 8 commits intonetdata:masterfrom
thiagoftsm:socket_tests
May 6, 2026
Merged

Unittest and socket collection#429
thiagoftsm merged 8 commits intonetdata:masterfrom
thiagoftsm:socket_tests

Conversation

@thiagoftsm
Copy link
Copy Markdown
Contributor

@thiagoftsm thiagoftsm commented May 5, 2026

Summary

Add unit tests and additional information to socket using buffer and arena.

Test Plan
  1. Get binaries according to your C library from [this](ADD ACTIONS LINK HERE) link and extract them inside a directory, for example: ../artifacts.
    You can also get everything for glibc [here](UPLOAD FILE WITH ALL BINARIES TO SIMPLIFY REVIEWERS).

  2. Extract them running:

    $ for i in `ls *.zip`; do unzip $i; rm .gitkeep ; rm $i; done
    $ for i in `ls *.tar`; do tar -xf $i; rm $i* ; done
  3. Compile branch an run the following tests:

    # make clean; make tester
    # for i in `seq 0 3`; do ./tests/legacy_test --netdata-path ../artifacts --content --iteration 1 --pid $i --log-path file_c_pid$i.txt; done
    # for i in `seq 0 3`; do ./gotests/go_tester --netdata-path ../artifacts --content --iteration 1 --pid $i --log-path file_go_pid$i.txt; done
  4. Every test should ends with Success, unless you do not have a specific target (function) available.

Additional information

This PR was tested on:

Linux Distribution Environment Kernel Version
Slackware current Bare metal 6.18.26
Arch Linux VM 7.0.3-arch1-2
Debian 13 VM 6.12.74
Alma 9 VM 5.14.0-611
Alma 8 VM 4.18.0-553
Slackware Current VM 5.15.203
Slackware Current VM 5.10.253

@thiagoftsm thiagoftsm mentioned this pull request May 5, 2026
@thiagoftsm thiagoftsm marked this pull request as ready for review May 6, 2026 01:03
@thiagoftsm thiagoftsm requested a review from Ferroin as a code owner May 6, 2026 01:03
@thiagoftsm thiagoftsm merged commit 3bfe3b7 into netdata:master May 6, 2026
40 checks passed
@thiagoftsm thiagoftsm deleted the socket_tests branch May 6, 2026 01:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant